Skip to content

fix(api_core): finalize during close of 'ResumableBidiRpc'.#9337

Merged
tseaver merged 3 commits intogoogleapis:masterfrom
tseaver:8616-9008-api_core-finalize-on-bidi-close
Oct 2, 2019
Merged

fix(api_core): finalize during close of 'ResumableBidiRpc'.#9337
tseaver merged 3 commits intogoogleapis:masterfrom
tseaver:8616-9008-api_core-finalize-on-bidi-close

Conversation

@tseaver
Copy link
Copy Markdown
Contributor

@tseaver tseaver commented Sep 27, 2019

Avoid blocking for ill-behaved daemon threads during BiDi shutdown.

Closes #8616, #9008.

@tseaver tseaver added api: pubsub Issues related to the Pub/Sub API. api: core api: firestore Issues related to the Firestore API. labels Sep 27, 2019
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Sep 27, 2019
Avoid blocking for ill-behaved daemon threads during BiDi shutdown.

Closes #8616, #9008.
Comment thread api_core/google/api_core/bidi.py
Comment thread api_core/google/api_core/bidi.py
@tseaver
Copy link
Copy Markdown
Contributor Author

tseaver commented Oct 1, 2019

@crwilcox PTAL

@crwilcox
Copy link
Copy Markdown
Contributor

crwilcox commented Oct 2, 2019

LGTM, but this will need to be rewritten to conform to conventionalcommits

@tseaver
Copy link
Copy Markdown
Contributor Author

tseaver commented Oct 2, 2019

@crwilcox

LGTM, but this will need to be rewritten to conform to conventionalcommits

I'll fix that in the merge commit.

@tseaver tseaver merged commit f068eb8 into googleapis:master Oct 2, 2019
@tseaver tseaver deleted the 8616-9008-api_core-finalize-on-bidi-close branch October 2, 2019 22:15
busunkim96 pushed a commit to busunkim96/google-cloud-python that referenced this pull request Oct 3, 2019
…s#9337)

Avoid blocking for ill-behaved daemon threads during BiDi shutdown.

Closes googleapis#8616, googleapis#9008.
parthea pushed a commit that referenced this pull request Nov 24, 2025
Avoid blocking for ill-behaved daemon threads during BiDi shutdown.

Closes #8616, #9008.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: core api: firestore Issues related to the Firestore API. api: pubsub Issues related to the Pub/Sub API. cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

PubSub: deadlock when cancelling a subscribe call

3 participants